Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(core): allow user updates #3648

Closed
wants to merge 4 commits into from
Closed

feat(core): allow user updates #3648

wants to merge 4 commits into from

Conversation

balazsorban44
Copy link
Member

Fixes #2269

TODO

  • Explain caveats (DB only, user must exist in session)
  • Docs (new warnings/errors)
  • Future improvements (update session?)

@github-actions github-actions bot added the core Refers to `@auth/core` label Jan 16, 2022
@balazsorban44 balazsorban44 temporarily deployed to Preview January 16, 2022 22:05 Inactive
@codecov-commenter
Copy link

codecov-commenter commented Jan 16, 2022

Codecov Report

Merging #3648 (e9f071c) into main (5998526) will decrease coverage by 0.19%.
The diff coverage is 26.82%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3648      +/-   ##
==========================================
- Coverage   13.08%   12.89%   -0.20%     
==========================================
  Files          91       91              
  Lines        1444     1474      +30     
  Branches      384      395      +11     
==========================================
+ Hits          189      190       +1     
- Misses       1241     1270      +29     
  Partials       14       14              
Impacted Files Coverage Δ
src/core/index.ts 0.00% <0.00%> (ø)
src/core/routes/session.ts 0.00% <0.00%> (ø)
src/lib/logger.ts 64.28% <ø> (ø)
src/react/index.tsx 87.24% <39.28%> (-11.23%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5998526...e9f071c. Read the comment docs.

@github-actions
Copy link

github-actions bot commented Jan 16, 2022

🎉 Experimental release published on npm!

npm i next-auth@0.0.0-pr.3648.f1ad5ec6
yarn add next-auth@0.0.0-pr.3648.f1ad5ec6

@balazsorban44 balazsorban44 temporarily deployed to Preview January 16, 2022 23:58 Inactive
@balazsorban44 balazsorban44 temporarily deployed to Preview January 17, 2022 13:39 Inactive
@stale
Copy link

stale bot commented Mar 19, 2022

It looks like this issue did not receive any activity for 60 days. It will be closed in 7 days if no further activity occurs. If you think your issue is still relevant, commenting will keep it open. Thanks!

@stale stale bot added the stale Did not receive any activity for 60 days label Mar 19, 2022
@stale
Copy link

stale bot commented Mar 26, 2022

To keep things tidy, we are closing this issue for now. If you think your issue is still relevant, leave a comment and we might reopen it. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Refers to `@auth/core` stale Did not receive any activity for 60 days
Projects
None yet
Development

Successfully merging this pull request may close these issues.

make updating session easier
2 participants